fix: Use timezone.utc instead of UTC #6

MyPyDavid · 2025-12-12T12:51:39Z

I got an error cannot import name 'UTC' from 'datetime with py310

MyPyDavid · 2025-12-17T10:08:20Z

rdmo_chatbot/chatbot/adapter.py

        cl.user_session.set("project_id", project_id)

        # get lang_code from the copilot and store it in the session
-        lang_code = await self.call_copilot("getLangCode", default="en")


this returned a None object at the start of chat or something like that, so that lang_code was None

MyPyDavid · 2025-12-19T10:08:16Z

rdmo_chatbot/chatbot/adapter.py

        if store.has_history(user.identifier, project_id):
            content = getattr(config, f"CONTINUATION_{lang_code.upper()}", "")
            await cl.Message(content=content).send()
+            history = store.get_history(user.identifier, project_id)


I've added here the chat history to the messages below this continuation message. Added it as a feature here but would make sense right?

I thought about this, I am not sure. I need to check how it looks.

I like it, to be able to see it at least, but it may also get too long..

There is no distinction between bot messages and human messages though:

it only shows at hover over, and there must be a way to change the icons right?

MyPyDavid · 2025-12-19T10:09:13Z

rdmo_chatbot/plugin/static/chatbot/js/copilot.js

-      subject: subjectInput.value,
-      message: messageInput.value
-    }
+  $(submitButton).off('click').on('click', async () => {


the submit button for send email was hanging(didn't do anything) for me at some point.

jochenklar

Maybe we talk about the changes later this week, I also implemented the other things we discussed, so this needs a rebase.

jochenklar · 2026-01-06T12:03:43Z

README.md


 MIDDLEWARE.append('rdmo_chatbot.plugin.middleware.ChatbotMiddleware')

+# Address of the Chainlit server serving the copilot widget. Provide an


I don't think that will work since the cookie with chatbot_token will hot work.

ok, I thought it worked before but I actually get this in the deployment

Error: Unauthorized vce https:/.../chatbot/copilot/index.js:376

so it probably doesn't work..

jochenklar · 2026-01-06T12:06:17Z

rdmo_chatbot/chatbot/adapter.py

+
+        try:
+            result = await cl.CopilotFunction(name=name, args=kwargs).acall()
+        except Exception:


This is not a good pattern, was there a specific reason to add this? I am ok with the args -> kwargs, although now it is confusing in a different way.

jochenklar · 2026-01-06T12:08:07Z

rdmo_chatbot/chatbot/adapter.py

        return response_message

+    async def send_history(self, history, user):
+        assistant_name = getattr(config, "ASSISTANT_NAME", "Assistant")


just use config.ASSISTANT_NAME there should be a default.

jochenklar · 2026-01-06T12:08:44Z

rdmo_chatbot/chatbot/adapter.py

        if store.has_history(user.identifier, project_id):
            content = getattr(config, f"CONTINUATION_{lang_code.upper()}", "")
            await cl.Message(content=content).send()
+            history = store.get_history(user.identifier, project_id)


I thought about this, I am not sure. I need to check how it looks.

jochenklar · 2026-01-06T12:09:36Z

rdmo_chatbot/plugin/static/chatbot/css/copilot.css

    height: 2em;
 }

+.sr-only {


jochenklar · 2026-01-06T12:12:10Z

rdmo_chatbot/plugin/static/chatbot/js/copilot.js

  return string.length > maxLength ? string.slice(0, maxLength) + '…' : string;
 }

+const getCookie = (name) => {


This is not needed Cookies is in the window object now.

jochenklar · 2026-01-06T12:12:56Z

rdmo_chatbot/plugin/static/chatbot/js/copilot.js


-    $(contactModal).modal('hide')
+      $(contactModal).modal('hide')
+    } catch (error) {


Is this really needed or is this just ChatGPT boilerplate?

jochenklar · 2026-01-06T12:14:18Z

rdmo_chatbot/plugin/static/chatbot/js/copilot.js

-    const shadow = copilot.shadowRoot
+const observedShadows = new WeakSet()
+
+const ensureDialogAccessibility = ({


The rest of the file needs refactoring or at least a reason why it is needed. This is just AI code.

jochenklar · 2026-01-06T12:14:57Z

rdmo_chatbot/plugin/templates/projects/project_interview.html

    <script src="{{ settings.CHATBOT_URL }}/copilot/index.js"></script>
    <script src="{% static 'chatbot/js/copilot.js' %}"></script>
    <script>
+        const configuredChatbotUrl = "{{ settings.CHATBOT_URL }}";


I don't think this is needed.

I wanted to set a relative url path for CHATBOT_URL = "/chatbot" and got an error in JS Failed to construct 'URL': Invalid URL but now I can't seem to reproduce this error so i'll revert it again..

jochenklar · 2026-01-06T12:20:00Z

rdmo_chatbot/plugin/utils.py

All but this change should be a different PR 🥴.

now Ive minimized the changes and only made 2 commits

Signed-off-by: David Wallace <[email protected]>

jochenklar

Thanks, we need to fix the messages part.

MyPyDavid commented Dec 17, 2025

View reviewed changes

MyPyDavid commented Dec 19, 2025

View reviewed changes

jochenklar requested changes Jan 6, 2026

View reviewed changes

MyPyDavid force-pushed the fix-utc branch from e985ce0 to 77ac90c Compare January 14, 2026 10:27

MyPyDavid added 2 commits January 14, 2026 17:18

Add history to chat and config.ASSISTANT_NAME

0d76da8

Signed-off-by: David Wallace <[email protected]>

Fix timezone with timezone.utc

b5a81ca

Signed-off-by: David Wallace <[email protected]>

MyPyDavid force-pushed the fix-utc branch from 4d2f2c2 to b5a81ca Compare January 14, 2026 16:20

jochenklar requested changes Jan 15, 2026

View reviewed changes

jochenklar added 2 commits January 15, 2026 12:56

Add message_type to send_history

6e44427

Add separate send_continuation and send_confirmation methods

113ecaa

jochenklar approved these changes Jan 15, 2026

View reviewed changes


		MIDDLEWARE.append('rdmo_chatbot.plugin.middleware.ChatbotMiddleware')

		# Address of the Chainlit server serving the copilot widget. Provide an

fix: Use timezone.utc instead of UTC #6

Are you sure you want to change the base?

fix: Use timezone.utc instead of UTC #6

Conversation

MyPyDavid commented Dec 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MyPyDavid Dec 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jochenklar left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

jochenklar left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

MyPyDavid commented Dec 12, 2025 •

edited

Loading

MyPyDavid Dec 17, 2025 •

edited

Loading